Closed
Bug 1414764
Opened 8 years ago
Closed 8 years ago
Remove Mozilla specific options to avoid build warnings with gcc and -Wcomment enabled
Categories
(Developer Infrastructure :: Source Code Analysis, enhancement)
Developer Infrastructure
Source Code Analysis
Tracking
(firefox58 fixed)
RESOLVED
FIXED
mozilla58
Tracking | Status | |
---|---|---|
firefox58 | --- | fixed |
People
(Reporter: andi, Assigned: andi)
References
Details
Attachments
(1 file)
Formatting some comments in macro causes a -Wcomment errors like:
from:
----
>>#define MAX_COLSPAN 1000 // limit as IE and opera do. If this ever changes,
>> // change COL_SPAN_OFFSET/COL_SPAN_SHIFT accordingly.
----
to:
----
>>#define MAX_COLSPAN \
>> 1000 // limit as IE and opera do. If this ever changes, \
>> // change COL_SPAN_OFFSET/COL_SPAN_SHIFT accordingly.
----
warning treated:
>> warning: multi-line comment [-Wcomment]
The latest version of clang-format fixes the ReflowComments and also CommentPragmas options that cause this issue.
I strongly think that we should permit ReflowComments and disallow the skipping of comment formatting by removing CommentPragmas.
Updated•8 years ago
|
Summary: Remove Mozilla specific options to avoid build warnings with gcc trunk and -Wcomment enabled → Remove Mozilla specific options to avoid build warnings with gcc and -Wcomment enabled
Comment hidden (mozreview-request) |
Comment 2•8 years ago
|
||
mozreview-review |
Comment on attachment 8925463 [details]
Bug 1414764 - Remove Mozilla specific options to avoid build warnings with gcc and -Wcomment enabled.
https://reviewboard.mozilla.org/r/196586/#review201808
Attachment #8925463 -
Flags: review?(sledru) → review+
Comment 3•8 years ago
|
||
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.
hg error in cmd: hg rebase -s 4da9b8415b53 -d f43442328ed9: rebasing 432239:4da9b8415b53 "Bug 1414764 - Remove Mozilla specific options to avoid build warnings with gcc and -Wcomment enabled. r=sylvestre" (tip)
merging .clang-format
warning: conflicts while merging .clang-format! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)
Comment 4•8 years ago
|
||
We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.
hg error in cmd: hg rebase -s 4da9b8415b53 -d f43442328ed9: rebasing 432239:4da9b8415b53 "Bug 1414764 - Remove Mozilla specific options to avoid build warnings with gcc and -Wcomment enabled. r=sylvestre" (tip)
merging .clang-format
warning: conflicts while merging .clang-format! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)
Comment hidden (mozreview-request) |
Pushed by bpostelnicu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c9e777a6f988
Remove Mozilla specific options to avoid build warnings with gcc and -Wcomment enabled. r=sylvestre
Comment 7•8 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
Updated•7 years ago
|
Product: Core → Firefox Build System
Updated•3 years ago
|
Product: Firefox Build System → Developer Infrastructure
You need to log in
before you can comment on or make changes to this bug.
Description
•